Skip to content

@W-22821836@ Sync payment-instrument amount before createOrder so OMS can ingest#3875

Merged
sf-jie-dai merged 1 commit into
feature/264-order-managementfrom
develop
Jun 16, 2026
Merged

@W-22821836@ Sync payment-instrument amount before createOrder so OMS can ingest#3875
sf-jie-dai merged 1 commit into
feature/264-order-managementfrom
develop

Conversation

@sf-jie-dai

@sf-jie-dai sf-jie-dai commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Description

Types of Changes

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation update
  • Breaking change (could cause existing functionality to not work as expected)
  • Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

  • Removing a public function or component or prop
  • Adding a required argument to a function
  • Changing the data type of a function parameter or return value
  • Adding a new peer dependency to package.json

Changes

  • (change1)

How to Test-Drive This PR

  • (step1)

Checklists

General

  • Changes are covered by test cases
  • CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance

You must check off all items in one of the follow two lists:

  • There are no changes to UI

or...

Localization

  • Changes include a UI text update in the Retail React App (which requires translation)

@sf-jie-dai
@W-22952388@ Sync payment-instrument amount before createOrder so OMS…
1826895 
… can ingest (#3866)

@W-22952388@ Sync payment-instrument amount before createOrder so OMS can ingest (#3866)
@sf-jie-dai sf-jie-dai requested a review from a team as a code owner June 16, 2026 15:32
@git2gus

git2gus Bot commented Jun 16, 2026

Copy link
Copy Markdown

Git2Gus App is installed but the .git2gus/config.json doesn't have right values. You should add the required configuration.

@cc-prodsec

cc-prodsec commented Jun 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@sf-jie-dai sf-jie-dai changed the title @W-22952388@ Sync payment-instrument amount before createOrder so OMS can ingest @W-22821836@ Sync payment-instrument amount before createOrder so OMS can ingest Jun 16, 2026
@sf-jie-dai sf-jie-dai self-assigned this Jun 16, 2026
@sf-jie-dai sf-jie-dai merged commit 3f62852 into feature/264-order-management Jun 16, 2026
238 of 278 checks passed
@jeremy-jung1 jeremy-jung1 temporarily deployed to extra-features-e2e June 17, 2026 08:42 — with GitHub Actions Inactive
sf-madhuri-uppu pushed a commit that referenced this pull request Jun 17, 2026
@sf-jie-dai @sf-madhuri-uppu
@W-22952388@ Sync payment-instrument amount before createOrder so OMS…
f7c3f9c 
… can ingest(#3875)

@W-22952388@ Sync payment-instrument amount before createOrder so OMS can ingest
@jeremy-jung1 jeremy-jung1 temporarily deployed to extra-features-e2e June 18, 2026 08:31 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sf-madhuri-uppu sf-madhuri-uppu sf-madhuri-uppu approved these changes

@sf-shikhar-prasoon sf-shikhar-prasoon sf-shikhar-prasoon approved these changes

Assignees

@sf-jie-dai sf-jie-dai

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sf-jie-dai @cc-prodsec @sf-madhuri-uppu @sf-shikhar-prasoon @jeremy-jung1